Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix/66 gracefull crash handling #604

Merged
merged 12 commits into from
May 10, 2024

Conversation

AssemblyJohn
Copy link
Contributor

@AssemblyJohn AssemblyJohn commented May 3, 2024

Describe your changes

Interface update for CSR requests and other libevse-security crash handlings.

Issue ticket number and link

EVerest/libevse-security#70

Supersedes: #596 as it includes all changes made there.

Checklist before requesting a review

Signed-off-by: AssemblyJohn <[email protected]>
Signed-off-by: AssemblyJohn <[email protected]>
Signed-off-by: AssemblyJohn <[email protected]>
Signed-off-by: AssemblyJohn <[email protected]>
@AssemblyJohn AssemblyJohn force-pushed the bugfix/66-gracefull-crash-handling branch from 30665ea to 116a122 Compare May 3, 2024 10:01
include/ocpp/common/evse_security.hpp Outdated Show resolved Hide resolved
lib/ocpp/v201/charge_point.cpp Show resolved Hide resolved
lib/ocpp/v16/charge_point_impl.cpp Show resolved Hide resolved
@Pietfried Pietfried self-assigned this May 6, 2024
@AssemblyJohn AssemblyJohn force-pushed the bugfix/66-gracefull-crash-handling branch from a4efe1e to 422be31 Compare May 6, 2024 12:49
Signed-off-by: AssemblyJohn <[email protected]>
@AssemblyJohn AssemblyJohn force-pushed the bugfix/66-gracefull-crash-handling branch from 422be31 to 3055563 Compare May 6, 2024 12:52
@AssemblyJohn
Copy link
Contributor Author

Comments have been addressed, please, re-review.

@AssemblyJohn AssemblyJohn requested a review from Pietfried May 6, 2024 12:53
lib/ocpp/common/evse_security_impl.cpp Outdated Show resolved Hide resolved
include/ocpp/common/evse_security_impl.hpp Outdated Show resolved Hide resolved
lib/ocpp/common/websocket/websocket_libwebsockets.cpp Outdated Show resolved Hide resolved
lib/ocpp/common/websocket/websocket_tls.cpp Outdated Show resolved Hide resolved
Signed-off-by: AssemblyJohn <[email protected]>
@AssemblyJohn AssemblyJohn force-pushed the bugfix/66-gracefull-crash-handling branch from 0f4f8f1 to e68e61f Compare May 7, 2024 12:37
Signed-off-by: AssemblyJohn <[email protected]>
@AssemblyJohn
Copy link
Contributor Author

Comments implemented, re-review required.

@AssemblyJohn AssemblyJohn requested a review from Pietfried May 7, 2024 12:46
include/ocpp/common/evse_security.hpp Outdated Show resolved Hide resolved
include/ocpp/common/evse_security.hpp Outdated Show resolved Hide resolved
include/ocpp/common/evse_security.hpp Outdated Show resolved Hide resolved
lib/ocpp/common/websocket/websocket_libwebsockets.cpp Outdated Show resolved Hide resolved
lib/ocpp/common/websocket/websocket_tls.cpp Outdated Show resolved Hide resolved
lib/ocpp/common/websocket/websocket_libwebsockets.cpp Outdated Show resolved Hide resolved
lib/ocpp/common/websocket/websocket_tls.cpp Outdated Show resolved Hide resolved
Signed-off-by: AssemblyJohn <[email protected]>
Signed-off-by: AssemblyJohn <[email protected]>
@AssemblyJohn
Copy link
Contributor Author

Comments solved, request re-review.

@AssemblyJohn AssemblyJohn requested a review from Pietfried May 8, 2024 07:57
Signed-off-by: AssemblyJohn <[email protected]>
@AssemblyJohn AssemblyJohn merged commit cea4087 into main May 10, 2024
3 of 4 checks passed
@AssemblyJohn AssemblyJohn deleted the bugfix/66-gracefull-crash-handling branch May 10, 2024 09:22
hikinggrass added a commit that referenced this pull request May 10, 2024
When #604 was merged it did not point to a git hash on the main branch but on a now deleted feature branch

Signed-off-by: Kai-Uwe Hermann <[email protected]>
hikinggrass added a commit that referenced this pull request May 10, 2024
When #604 was merged it did not point to a git hash on the main branch but on a now deleted feature branch

Signed-off-by: Kai-Uwe Hermann <[email protected]>
christopher-davis-afs pushed a commit to US-JOET/libocpp that referenced this pull request May 30, 2024
* Updated interface usage
* Updated dependencies for new interface
* Fixed mocks
* Updated deps
* Updated native interface usage
* Updated deps
* Updated for comments
* Implemented code comments
* Updated dependencies
* Updated some comments
* Update legacy websocket++
* Updated deps
---------

Signed-off-by: AssemblyJohn <[email protected]>
christopher-davis-afs pushed a commit to US-JOET/libocpp that referenced this pull request May 30, 2024
When EVerest#604 was merged it did not point to a git hash on the main branch but on a now deleted feature branch

Signed-off-by: Kai-Uwe Hermann <[email protected]>
drmrd pushed a commit to US-JOET/libocpp that referenced this pull request Jun 4, 2024
* Updated interface usage
* Updated dependencies for new interface
* Fixed mocks
* Updated deps
* Updated native interface usage
* Updated deps
* Updated for comments
* Implemented code comments
* Updated dependencies
* Updated some comments
* Update legacy websocket++
* Updated deps
---------

Signed-off-by: AssemblyJohn <[email protected]>
Signed-off-by: Daniel Moore <[email protected]>
drmrd pushed a commit to US-JOET/libocpp that referenced this pull request Jun 4, 2024
When EVerest#604 was merged it did not point to a git hash on the main branch but on a now deleted feature branch

Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Daniel Moore <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants